Skip to content

CI: rename GCP subnets used for integration specs#564

Merged
aramprice merged 1 commit into
ubuntu-jammyfrom
rename-integration-subnets
Apr 20, 2026
Merged

CI: rename GCP subnets used for integration specs#564
aramprice merged 1 commit into
ubuntu-jammyfrom
rename-integration-subnets

Conversation

@aramprice

@aramprice aramprice commented Apr 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Pipeline already reconfigured.

Copilot AI review requested due to automatic review settings April 20, 2026 18:56
@coderabbitai

coderabbitai Bot commented Apr 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 89358074-4cf9-4b79-b072-2df42e4a388b

📥 Commits

Reviewing files that changed from the base of the PR and between e73c878 and e079e55.

📒 Files selected for processing (2)
  • ci/pipelines/builder.yml
  • ci/pipelines/vars.yml

Walkthrough

This change updates GCP infrastructure naming in pipeline configuration files. The ci/pipelines/builder.yml file has subnet naming conventions updated from bosh-integration- prefix to stemcell-builder-integration- prefix across multiple director deploy tasks and bats configuration parameters. Additionally, ci/pipelines/vars.yml updates the stemcell_details.subnet_int value from "3" to "22", with an inline comment indicating this should reflect the last two digits of the release year.

Suggested reviewers

  • rkoster
  • benjaminguttmann-avtq
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a description explaining the purpose and impact of the GCP subnet renaming changes to provide context for reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: renaming GCP subnet naming conventions from 'bosh-integration-...' to 'stemcell-builder-integration-...' across pipeline configuration files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rename-integration-subnets

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aramprice aramprice merged commit b62f580 into ubuntu-jammy Apr 20, 2026
16 checks passed
@aramprice aramprice deleted the rename-integration-subnets branch April 20, 2026 18:58
Copilot AI reviewed Apr 20, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Renames the GCP subnets used by the CI integration/bats pipelines to a new stemcell-builder-integration-* naming scheme, and updates the subnet index used to derive CIDRs/IPs.

Changes:

  • Update stemcell_details.subnet_int from "3" to "22" (used for IP/CIDR derivation).
  • Rename GCP_SUBNET_NAME / VARS_SUBNETWORK_DEFAULT values from bosh-integration-* to stemcell-builder-integration-* (including ipv6 variant).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ci/pipelines/vars.yml Updates the subnet index (subnet_int) used across GCP integration jobs.
ci/pipelines/builder.yml Renames the GCP subnet names referenced by integration and BATs jobs (ipv4 + ipv6).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci/pipelines/vars.yml
Comment thread ci/pipelines/builder.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aramprice